Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

I18N - Home #322

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

I18N - Home #322

wants to merge 10 commits into from

Conversation

MoisesMsa
Copy link
Collaborator

@MoisesMsa MoisesMsa commented Nov 18, 2024

Português (BR) | English (US)

Comunidade

Tipo de alteração

  • 🐞 Correção de problema
  • ✨ Melhoria ou nova funcionalidade
  • 📰 Nova postagem no blog

Issues relacionadas

#163

Validação

  • Validei a alteração no link gerado pelo bot da Netlify (Deploy Preview/Preview on mobile)
  • Validei o Layout responsivo (desktop/mobile) após a implementação
  • Verifiquei o registro do deploy (Latest deploy log) e nenhum novo alerta ou erro foi adicionado

Evidências

Anexe evidências do antes e do depois da alteração (quando necessário).

Alterações:

  • Adiciona as bibliotecas @ngx-translate/core e @ngx-translate/http-loader
  • Adiciona um serviço para i18n
  • Altera o header para adicionar um language switcher (reaproveitado da PR I18n poc #165 )
  • Adiciona um arquivo global de localization para o header/footer
  • Adiciona um arquivo de localization para a home e seus componentes
  • Atualiza o componente da home para exibir as strings vindas do localization file

home_i18n

Documentação

  • A documentação deste repositório foi atualizada (quando necessário).
  • Esta alteração requer que a documentação externa seja atualizada.

Copy link

netlify bot commented Nov 18, 2024

Deploy Preview for querido-diario-plataforma ready!

Name Link
🔨 Latest commit 6a267ad
🔍 Latest deploy log https://app.netlify.com/sites/querido-diario-plataforma/deploys/673ff49f28435a000872e08d
😎 Deploy Preview https://deploy-preview-322--querido-diario-plataforma.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@MoisesMsa
Copy link
Collaborator Author

Abri a PR porque está bem grande, como a #165 estava desatualizada a um tempo criei uma nova branch e optei por outra abordagem já que a maior parte das strings já estão mapeadas em um JSON (preciso fazer algumas melhorias quanto a isso pq algums arquivos compartilham infomações de ícones, estilos que precisei copiar pros arquivos localization para não quebrar a página).
A ideia é criar as outras PRs a partir dessa com a tradução das outras páginas e dar uma revisada se as traduções estão boas porque acho que alguns termos precisam serem melhores adptados.

Copy link
Collaborator

@lguima lguima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

muito bom, @MoisesMsa! excelente solução! 👏

reparei apenas que a gente traduziu a rota também, mas ela não existe em inglês, ai o link do cabeçalho fica quebrado

outro ponto é o layout mobile, foi intencional não cobrir nessa PR?

src/assets/i18n/en/global.json Outdated Show resolved Hide resolved
@MoisesMsa
Copy link
Collaborator Author

@lguima ajustei, as rotas de EN para as atuais em PT

Copy link
Collaborator

@lguima lguima left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🚀

@ogecece
Copy link
Member

ogecece commented Dec 11, 2024

Gente, acho que já podemos mesclar e criar as issues para as partes que faltam (mobile, outras páginas, etc.). A solução tá excelente! 1000 parabéns proceis! 1000 é pouco, aliás: https://www.youtube.com/watch?v=1Mcdh2Vf2Xk

@lguima vc poderia atualizar essa branch com rebase pra pegar a main atualizada, mesclar e criar as issues?

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants